Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete sheet1 content before writing #25

Merged
merged 3 commits into from
Oct 25, 2024

Conversation

Alex-Monahan
Copy link
Contributor

@Alex-Monahan Alex-Monahan commented Oct 24, 2024

Up to you if you want to merge! This will change the append behavior to always clear out the entire sheet.

Edit: This uses another API I found that uses the range style API, so it refers back to the same hardcoded sheet_name ("Sheet1"), so it should work without needing to find a sheetId!

@Alex-Monahan
Copy link
Contributor Author

Ah, shoot, forgot to run the test suite. I can do that beforehand next time! This might only be useful as a code snippet anyway (might not be mergeable yet anyway)

@archiewood
Copy link
Member

Ideal! - Good sleuthing - I can test locally

@archiewood archiewood merged commit a384fdf into evidence-dev:main Oct 25, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants